Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make Makefile a little easier to use #1069

Merged
merged 3 commits into from
Oct 10, 2020
Merged

Conversation

rhatdan
Copy link
Member

@rhatdan rhatdan commented Oct 8, 2020

By default we should build bin/skopeo locally
and build docs locally.

Show output when doing make docs.

Add description in make help to explain default
behaviour.

Fixes: #580

Signed-off-by: Daniel J Walsh [email protected]

@rhatdan
Copy link
Member Author

rhatdan commented Oct 8, 2020

@vrothberg @mtrmac @QiWang19 PTAL

Copy link
Contributor

@mtrmac mtrmac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK, but install.md needs to be updated as well.

By default we should build bin/skopeo locally
and build docs locally.

Show output when doing make docs.

Add description in `make help` to explain default
behaviour.

Signed-off-by: Daniel J Walsh <[email protected]>
@rhatdan
Copy link
Member Author

rhatdan commented Oct 8, 2020

Closes: #755

@rhatdan
Copy link
Member Author

rhatdan commented Oct 8, 2020

install.md Outdated Show resolved Hide resolved
install.md Outdated Show resolved Hide resolved
install.md Outdated Show resolved Hide resolved
rhatdan and others added 2 commits October 9, 2020 06:57
Co-authored-by: Miloslav Trmač <[email protected]>
Co-authored-by: Tom Sweeney <[email protected]>
Copy link
Contributor

@mtrmac mtrmac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Any idea why I can’t restart the Travis tests?

@rhatdan
Copy link
Member Author

rhatdan commented Oct 9, 2020

@mtrmac nope.

@vrothberg
Copy link
Member

@mtrmac nope.

Just restarted it but I had to sign in.

Copy link
Member

@vrothberg vrothberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Let's merge when it turned green

@QiWang19
Copy link
Contributor

QiWang19 commented Oct 9, 2020

LGTM

1 similar comment
@TomSweeneyRedHat
Copy link
Member

LGTM

@TomSweeneyRedHat
Copy link
Member

/lgtm

@rhatdan rhatdan merged commit c18a977 into containers:master Oct 10, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Confusing make targets
5 participants